Skip to content

allow hyphens in database name#2815

Open
JeffResc wants to merge 1 commit intozalando:masterfrom
JeffResc:databasename-regex
Open

allow hyphens in database name#2815
JeffResc wants to merge 1 commit intozalando:masterfrom
JeffResc:databasename-regex

Conversation

@JeffResc
Copy link
Copy Markdown

Allow for hyphens in the database name, they are currently supported by Postgres, but not this operator

Current behavior:

time="2024-12-10T22:58:31Z" level=info msg="database \"my-database\" has invalid name" cluster-name=postgres/pg-cluster pkg=cluster

@FxKu
Copy link
Copy Markdown
Member

FxKu commented Dec 18, 2024

@JeffResc thanks for your PR. I would prefer to make the database regex configurable like it was attempted in #2139. Maybe you can pick up the work done there?

@FxKu FxKu added the minor label Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants